Skip to content

Conversation

steveklabnik
Copy link
Contributor

There's one big one lift, I'm filing a bug for it soon.

r? @rust-lang/docs

@steveklabnik
Copy link
Contributor Author

cc @alexcrichton who noticed this

@steveklabnik
Copy link
Contributor Author

Other bug is #45420

@QuietMisdreavus
Copy link
Contributor

r=me pending travis. apparently the footnote issue was fixed in a later version of pulldown, so it's just a matter of updating rustdoc to point at a version that contains that fix. I opened #45421 to do that.

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 20, 2017
@QuietMisdreavus
Copy link
Contributor

The travis logs confirm that the only warnings left in std are connected to the footnote in libcore's crate-root docs. After double-checking against the latest commit in #45421, i'm reasonably sure that that should grab those warnings.

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Oct 20, 2017

📌 Commit f8f9005 has been approved by QuietMisdreavus

@@ -22,7 +22,7 @@ use mem;
/// This is currently the default hashing function used by standard library
/// (eg. `collections::HashMap` uses it by default).
///
/// See: https://131002.net/siphash/
/// See: <https://131002.net/siphash>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpick: inconsistent trailing slash.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

both work

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The version without the trailing slash results in a 301 redirect.

@kennytm kennytm added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 21, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Oct 21, 2017
…ngs, r=QuietMisdreavus

Fix most rendering warnings from switching to CommonMark

There's one big one lift, I'm filing a bug for it soon.

r? @rust-lang/docs
bors added a commit that referenced this pull request Oct 21, 2017
Rollup of 6 pull requests

- Successful merges: #45227, #45356, #45407, #45411, #45418, #45419
- Failed merges: #45421
@bors bors merged commit f8f9005 into rust-lang:master Oct 21, 2017
@steveklabnik steveklabnik deleted the fix-commonmark-renderings branch October 25, 2017 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants